search for: localvarid

...ngly), although I'm not sure a prefix $ is the best syntax. Maybe we could even get away with %42: as a BB label and that would be maximally reminiscent. The way that numbered local variables are handled is sort of ad-hoc (it is actually also handled in the Lexer; all the parser sees is lltok::LocalVarID). By just changing LLLexer::LexPercent in LLLexer.cpp to recognize a local label and emit a "LocalLabelID" token, then adding an `else if` to the first `if` in LLParser::ParseBasicBlock, you could probably get a working solution too. However, this introduces an inconsistency in that now t...

...not sure a prefix $ is the best syntax. > > Maybe we could even get away with %42: as a BB label and that would be > maximally reminiscent. The way that numbered local variables are handled is > sort of ad-hoc (it is actually also handled in the Lexer; all the parser > sees is lltok::LocalVarID). By just changing LLLexer::LexPercent in > LLLexer.cpp to recognize a local label and emit a "LocalLabelID" token, > then adding an `else if` to the first `if` in LLParser::ParseBasicBlock, > you could probably get a working solution too. However, this introduces an > inconsi...

...4,7 +284,7 @@ // Handle GlobalVarID: @[0-9]+ if (isdigit(CurPtr[0])) { - for (++CurPtr; isdigit(CurPtr[0]); ++CurPtr); + for (++CurPtr; isdigit(CurPtr[0]); ++CurPtr) ; uint64_t Val = atoull(TokStart+1, CurPtr); if ((unsigned)Val != Val) @@ -335,7 +335,7 @@ // Handle LocalVarID: %[0-9]+ if (isdigit(CurPtr[0])) { - for (++CurPtr; isdigit(CurPtr[0]); ++CurPtr); + for (++CurPtr; isdigit(CurPtr[0]); ++CurPtr) ; uint64_t Val = atoull(TokStart+1, CurPtr); if ((unsigned)Val != Val) @@ -717,7 +717,7 @@ // At this point, it is either a label, int or fp c...

The language reference states that local temporaries begin with index 0, but if I try that on my not-entirely-up-to-date v3.4 llc (it is like a week old), I get an error "instruction expected to be numbered '%1'". Also, quite a few examples in the LR uses %0 as a local identifier. Should I fix those or is it a problem in llc? -- Mikael -------------- next part --------------

...is the best syntax. >> >> Maybe we could even get away with %42: as a BB label and that would be >> maximally reminiscent. The way that numbered local variables are handled is >> sort of ad-hoc (it is actually also handled in the Lexer; all the parser >> sees is lltok::LocalVarID). By just changing LLLexer::LexPercent in >> LLLexer.cpp to recognize a local label and emit a "LocalLabelID" token, >> then adding an `else if` to the first `if` in LLParser::ParseBasicBlock, >> you could probably get a working solution too. However, this introduces an...

...while (isalnum(CurPtr[0]) || CurPtr[0] == '-' || CurPtr[0] == '$' || CurPtr[0] == '.' || CurPtr[0] == '_') ++CurPtr; - + llvmAsmlval.StrVal = new std::string(TokStart+1, CurPtr); // Skip % return LOCALVAR; } - + // Handle LocalVarID: %[0-9]+ if (isdigit(CurPtr[0])) { - for (++CurPtr; isdigit(CurPtr[0]); ++CurPtr); - + for (++CurPtr; isdigit(CurPtr[0]); ++CurPtr) + ; + uint64_t Val = atoull(TokStart+1, CurPtr); if ((unsigned)Val != Val) GenerateError("Invalid value number (too large)!&quo...

...>>> >>> Maybe we could even get away with %42: as a BB label and that would be >>> maximally reminiscent. The way that numbered local variables are handled is >>> sort of ad-hoc (it is actually also handled in the Lexer; all the parser >>> sees is lltok::LocalVarID). By just changing LLLexer::LexPercent in >>> LLLexer.cpp to recognize a local label and emit a "LocalLabelID" token, >>> then adding an `else if` to the first `if` in LLParser::ParseBasicBlock, >>> you could probably get a working solution too. However, this in...

...;>>> Maybe we could even get away with %42: as a BB label and that would be >>>> maximally reminiscent. The way that numbered local variables are handled is >>>> sort of ad-hoc (it is actually also handled in the Lexer; all the parser >>>> sees is lltok::LocalVarID). By just changing LLLexer::LexPercent in >>>> LLLexer.cpp to recognize a local label and emit a "LocalLabelID" token, >>>> then adding an `else if` to the first `if` in LLParser::ParseBasicBlock, >>>> you could probably get a working solution too. Howe...