[llvm-commits] [llvm] r84020 - in /llvm/trunk: include/llvm/InlineAsm.h lib/AsmParser/LLLexer.cpp lib/AsmParser/LLParser.cpp lib/AsmParser/LLToken.h lib/Bitcode/Reader/BitcodeReader.cpp lib/Bitcode/Writer/BitcodeWriter.cpp lib/VMCore/AsmWriter.cpp lib/VMCore/Core.cpp lib/VMCore/InlineAsm.cpp test/Assembler/msasm.ll
clattner at apple.com
Tue Oct 13 23:01:08 PDT 2009
On Oct 13, 2009, at 6:28 PM, Dale Johannesen wrote:
>> In general I think that this is a great approach. One concern I
>> have is that "is ms asm" doesn't tell me what the flag does. Would
>> it be possible to name this something like "might be call" or
>> something that more accurately conveys the semantic effect of
>> this? I'd much prefer LangRef to have a concrete description of
>> what it does (like volatile) instead of saying "originally from an
>> asm block" which doesn't communicate what it does or when someone
>> would want to set it.
> Sure, we could do that; my intent was that if any other semantic
> differences show up they can also key off this flag instead of
> adding a new one. Someone would want to set it if their source uses
> asm blocks.
> In any event I am not going to get to changing this before I
> disappear for a few days.
No rush, thanks for working on this!
More information about the llvm-commits