#milkymist IRC log for Friday, 2012-07-27

wolfspraulhas anyone worked with xilinx' new vivado design suite yet?10:49
lekernelhi11:01
lekernelanyone knows the clang command line option that disable usage of the hw divider, barrel shifter, etc. ?11:02
lekernelthey are "SubtargetFeature" in LM32.td, but it's not well documented how this connects to the clang command line11:02
Action: lekernel looks at stekern11:02
stekernlekernel: -Xclang -target-feature -Xclang +feature11:09
stekernor -feature11:09
stekernto turn it off11:09
lekernelthanks!11:13
lekernelfor some reason divider/barrel/etc. switches are commented out in clang... hmm11:15
stekernyeah, I just saw that when I went looking in there to make sure you have setFeatureEnabled implemented11:16
lekernelof course, clang has to redefine all the accepted target features... they couldn't simply pull it automatically from LM32.td11:16
stekerndon't get me started on that...11:17
lekernelseems to work... or at least produces a convicing object file ;)11:49
lekerneltesting the code now...11:49
GitHub130[clang-lm32] sbourdeauducq pushed 1 new commit to master: http://git.io/n8SfTg11:51
GitHub130[clang-lm32/master] LM32: add options to enable/disable barrel shifter, multiplier and divider - Sebastien Bourdeauducq11:51
lekernelmeh. it doesn't. still tries to use the extensions...11:54
lekernelalso there's no subtarget feature for the sign extender11:57
stekerndoes it work if you do: clang -c -S file.c -emit-llvm -o - | llc -mattr=feature -o -12:01
stekernlooks like sign extending is implemented though, but you maybe want to be able to turn it off?12:05
stekern(assuming that you are speaking about sextb and sexth)12:07
lekernelyes, it can be turned off using CFG_SIGN_EXTEND_ENABLED in Verilog, but there's no option in clang/llvm to disable usage of the corresponding instructions12:07
lekernelwill dive into llvm a bit later... need to deliver some gateware + lm32 software for http://tunka-hrjrg.desy.de/e98279/index_eng.html by next week12:11
lekerneli'll use gcc for now12:14
stekernlekernel_: I took a peek at why the target features aren't working, they are initialised to true in the constructor, but the subtarget features can only be turned on when parsing them (a -feature will just omit setting it to true), so any features that should be on by default should be handled by the feature parsing23:28
stekernI've got a patch that fixes it, I'll just go ahead and commit that, yell if I rather should post to the ML before committing23:29
GitHub189[llvm-lm32] skristiansson pushed 1 new commit to master: http://git.io/n1n1JA23:38
GitHub189[llvm-lm32/master] Initialise subtarget features to false in constructor - Stefan Kristiansson23:38
--- Sat Jul 28 201200:00

Generated by irclog2html.py 2.9.2 by Marius Gedminas - find it at mg.pov.lt!