[PATCH] D65428: Remove cache for macro arg stringization

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 29 18:31:50 PDT 2019


rnk created this revision.
rnk added reviewers: vsk, rsmith.
Herald added a project: clang.

The cache recorded the wrong expansion location for all but the first
stringization. It seems uncommon to stringize the same macro argument
multiple times, so this cache doesn't seem that important.

Fixes PR39942


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D65428

Files:
  clang/include/clang/Lex/MacroArgs.h
  clang/lib/Lex/MacroArgs.cpp
  clang/lib/Lex/TokenLexer.cpp
  clang/test/CoverageMapping/macro-stringize-twice.cpp
  clang/unittests/Lex/LexerTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D65428.212261.patch
Type: text/x-patch
Size: 6670 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190730/4f36f2d6/attachment-0001.bin>


More information about the cfe-commits mailing list