[PATCH] D43006: MC: store defined macros in MCContext

[Rafael Avila de Espindola via llvm-commits]

Roger via Phabricator <reviews at reviews.llvm.org> writes:


> +#ifndef LLVM_MC_MCASMMACRO_H
> +#define LLVM_MC_MCASMMACRO_H
> +
> +#include "llvm/MC/MCParser/MCAsmLexer.h"
> +#include <cstddef>
> +#include <cstdint>
> +#include <string>
> +#include <vector>

I don't think you need all these in this header.

LGTM with that. Do you have commit access?

Cheers,
Rafael